Skip to content

Conversation

@jsoriano
Copy link
Member

@jsoriano jsoriano commented Oct 22, 2020

Cherry-pick of PR #21981 to 6.8 branch. Original message:

  • Bug

What does this PR do?

Updates the url of reviewdog.

Why is it important?

related to #21976
It looks that Go tries to install the reviewdog from github.com/haya14busa/reviewdog, but the reviewdog was moved to reviewdog organization.
New url of the reviewdog is github.com/reviewdog/reviewdog.

Checklist

  • My code follows the style guidelines of this project
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have made corresponding change to the default configuration files
  • I have added tests that prove my fix is effective or that my feature works
  • I have added an entry in CHANGELOG.next.asciidoc or CHANGELOG-developer.next.asciidoc.

Author's Checklist

  • [ ]

How to test this PR locally

Related issues

Use cases

Screenshots

Logs

@botelastic botelastic bot added the needs_team Indicates that the issue/PR needs a Team:* label label Oct 22, 2020
@jsoriano jsoriano added the Team:Integrations Label for the Integrations team label Oct 22, 2020
@elasticmachine
Copy link
Collaborator

Pinging @elastic/integrations (Team:Integrations)

@botelastic botelastic bot removed the needs_team Indicates that the issue/PR needs a Team:* label label Oct 22, 2020
@jsoriano
Copy link
Member Author

Linting failure is not related.

@jsoriano jsoriano merged commit 6741ca1 into elastic:6.8 Oct 22, 2020
@jsoriano jsoriano deleted the backport_21981_6.8 branch October 22, 2020 15:25
v1v added a commit to v1v/beats that referenced this pull request Oct 29, 2020
* upstream/6.8:
  Edit 6.8.13 release notes (elastic#22108)
  Cherry-pick elastic#21981 to 6.8: fix the url of reviewdog (elastic#22071)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport review Team:Integrations Label for the Integrations team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants